Skip to content

chore(css): use "brand" over "green"#8915

Open
avivkeller wants to merge 1 commit into
mainfrom
brand-green'
Open

chore(css): use "brand" over "green"#8915
avivkeller wants to merge 1 commit into
mainfrom
brand-green'

Conversation

@avivkeller
Copy link
Copy Markdown
Member

It's easier for overriders to understand the -brand- convention for the primary color, rather than -green-

Copilot AI review requested due to automatic review settings May 25, 2026 22:05
@avivkeller avivkeller requested a review from a team as a code owner May 25, 2026 22:05
@vercel
Copy link
Copy Markdown

vercel Bot commented May 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
nodejs-org Error Error May 25, 2026 10:14pm

Request Review

@cursor
Copy link
Copy Markdown

cursor Bot commented May 25, 2026

PR Summary

Low Risk
Cosmetic token/class rename with 1:1 color mapping; no logic or auth changes, though any external overrides still using green-* would need updating.

Overview
Renames the primary palette from green-* to brand-* across the design system and site, without changing the underlying hex values in theme.css (--color-brand-* replaces --color-green-*).

Tailwind/@apply usage is updated everywhere the old primary green appeared—buttons, tabs, sidebars, search, markdown links, alerts/badges, focus rings, gradients, and related Storybook swatches—so overrides can target a semantic brand token instead of a literal color name.

A few files only reorder @apply lines or split important modifiers (e.g. md:hidden); behavior should stay the same aside from the class rename.

Reviewed by Cursor Bugbot for commit 18c0d60. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR renames the UI primary color token/utility naming from green to brand to make the design system’s “primary color” convention clearer for downstream overrides.

Changes:

  • Renames Tailwind v4 @theme color tokens from --color-green-* to --color-brand-* and updates the custom glow utility to use the new token.
  • Updates @apply usage across UI component CSS modules from *-green-* utilities to *-brand-*.
  • Updates Storybook design color swatches and a couple of app/site CSS modules to use the new brand utilities.

Reviewed changes

Copilot reviewed 36 out of 36 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
packages/ui-components/src/styles/theme.css Renames theme color tokens to brand and updates glow utility token usage.
packages/ui-components/src/styles/markdown.css Switches markdown link styling utilities to brand.
packages/ui-components/src/Containers/Sidebar/SidebarItem/index.module.css Updates sidebar item hover/active styling utilities to brand.
packages/ui-components/src/Containers/NavBar/NavItem/index.module.css Updates active nav background utility to brand.
packages/ui-components/src/Containers/Footer/index.module.css Updates footer link colors to brand.
packages/ui-components/src/Common/ThemeToggle/index.module.css Updates highlighted/active item background utilities to brand.
packages/ui-components/src/Common/Tabs/index.module.css Updates active tab border/text utilities to brand.
packages/ui-components/src/Common/Switch/index.module.css Updates focus ring + checked background utilities to brand.
packages/ui-components/src/Common/SkipToContentButton/index.module.css Updates focus background/ring utilities to brand.
packages/ui-components/src/Common/Signature/SignatureItem/index.module.css Updates “return” background utility to brand.
packages/ui-components/src/Common/Signature/SignatureHeader/index.module.css Updates signature link colors to brand.
packages/ui-components/src/Common/Select/index.module.css Updates highlighted/hover background utilities to brand.
packages/ui-components/src/Common/Search/Suggestions/index.module.css Updates suggestion item text/focus border utilities to brand.
packages/ui-components/src/Common/Search/Results/Tabs/index.module.css Updates selected facet tab border utilities to brand.
packages/ui-components/src/Common/Search/Results/Hit/index.module.css Updates hit focus border utility to brand.
packages/ui-components/src/Common/Search/Chat/Trigger/index.module.css Updates chat trigger focus border utilities to brand.
packages/ui-components/src/Common/Search/Chat/Input/index.module.css Updates chat send button background utilities to brand.
packages/ui-components/src/Common/Search/Chat/Actions/index.module.css Updates selected action icon text utilities to brand.
packages/ui-components/src/Common/Preview/index.module.css Updates announcement gradient token utility to brand.
packages/ui-components/src/Common/LanguageDropDown/index.module.css Updates highlighted/current dropdown background utilities to brand.
packages/ui-components/src/Common/GlowingBackdrop/index.module.css Updates backdrop glow utilities to brand.
packages/ui-components/src/Common/DataTag/index.module.css Updates “property” tag background utility to brand.
packages/ui-components/src/Common/CodeTabs/index.module.css Updates active trigger border/text utilities to brand.
packages/ui-components/src/Common/ChangeHistory/index.module.css Updates hover/focus background utility to brand.
packages/ui-components/src/Common/Breadcrumbs/BreadcrumbLink/index.module.css Updates active breadcrumb background utilities to brand.
packages/ui-components/src/Common/Blockquote/index.module.css Updates blockquote border utilities to brand.
packages/ui-components/src/Common/BasePagination/PaginationListItem/index.module.css Updates current-page background utility to brand.
packages/ui-components/src/Common/BaseLinkTabs/index.module.css Updates active tab border/text utilities to brand.
packages/ui-components/src/Common/BaseButton/index.module.css Updates primary/glow button border/background utilities to brand.
packages/ui-components/src/Common/Banner/index.module.css Updates default banner background utility to brand.
packages/ui-components/src/Common/BadgeGroup/index.module.css Updates default badge group border/bg/text utilities to brand.
packages/ui-components/src/Common/Badge/index.module.css Updates default badge background utility to brand.
packages/ui-components/src/Common/AlertBox/index.module.css Updates success alert background/title utilities to brand.
packages/ui-components/design/colors.stories.tsx Updates color swatch story to use bg-brand-*.
apps/site/components/Common/LinkWithArrow/index.module.css Updates link-with-arrow colors to brand.
apps/site/components/Blog/BlogPostCard/index.module.css Updates blog post card subtitle color utilities to brand.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/ui-components/src/styles/theme.css
@codecov
Copy link
Copy Markdown

codecov Bot commented May 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.24%. Comparing base (b97f706) to head (18c0d60).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8915      +/-   ##
==========================================
+ Coverage   73.23%   73.24%   +0.01%     
==========================================
  Files         102      102              
  Lines        8628     8628              
  Branches      313      313              
==========================================
+ Hits         6319     6320       +1     
+ Misses       2308     2307       -1     
  Partials        1        1              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants